Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Print link to debugger UI #32

Closed
wants to merge 2 commits into from
Closed

Conversation

bjchambers
Copy link
Contributor

After successfully registering a debuggee, print a link to go to the
appropriate UI.

After successfully registering a debuggee, print a link to go to the
appropriate UI.
@bjchambers
Copy link
Contributor Author

R: @davorbonaci

@@ -424,6 +424,12 @@ public static DataflowPipelineRunner fromOptions(PipelineOptions options) {
return super.apply(new AssignWindows<>(transform), input);
}

private String debuggerMessage(String projectId, String uniqifier) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: uniquifier

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@davorbonaci
Copy link
Member

LGTM.

@asfgit asfgit closed this in db160fa Mar 23, 2016
davorbonaci added a commit to GoogleCloudPlatform/DataflowJavaSDK that referenced this pull request Mar 25, 2016
@bjchambers bjchambers deleted the cdbg2 branch November 21, 2016 21:42
echauchot added a commit to echauchot/beam that referenced this pull request May 12, 2017
cosmoskitten pushed a commit to cosmoskitten/beam that referenced this pull request Jun 16, 2017
query5: Add comment on key lifting (issue apache#30)

query10: Add comment for strange groupByKey (issue apache#31)

query11: Replace Count.perKey by Count.perElement (issue apache#32)
asfgit pushed a commit that referenced this pull request Aug 23, 2017
query5: Add comment on key lifting (issue #30)

query10: Add comment for strange groupByKey (issue #31)

query11: Replace Count.perKey by Count.perElement (issue #32)
tweise pushed a commit to tweise/beam that referenced this pull request Mar 26, 2018
Wire in output union tags by PTransform local output name
tvalentyn pushed a commit to tvalentyn/beam that referenced this pull request May 15, 2018
tvalentyn pushed a commit to tvalentyn/beam that referenced this pull request May 15, 2018
robertwb pushed a commit to robertwb/incubator-beam that referenced this pull request Apr 30, 2020
* Migrated Runners page

* Migrated Contribute page

* Refactored redirected links
robertwb pushed a commit to robertwb/incubator-beam that referenced this pull request Apr 30, 2020
* Migrated Runners page

* Migrated Contribute page

* Refactored redirected links
ajothomas referenced this pull request in ajothomas/beam Jun 28, 2022
sjvanrossum pushed a commit to sjvanrossum/beam that referenced this pull request May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants